Conversation
Added LOG_DIR environment variable for logging.
p-
requested review from
JarLob,
kevinbackhouse,
m-y-mo and
sylwia-budzynska
as code owners
There was a problem hiding this comment.
Pull request overview
This PR adds the LOG_DIR environment variable to the README.md documentation. The variable is set to /app/logs and appears alongside other environment variables like MEMCACHE_STATE_DIR, CODEQL_DBS_BASE_PATH, and DATA_DIR that are used to configure directory paths for the SecLab Taskflow Agent.
- Adds LOG_DIR=/app/logs to the environment variables configuration example
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| LOG_DIR=/app/logs | ||
| ``` | ||
|
|
There was a problem hiding this comment.
The LOG_DIR environment variable is added to the configuration example but is not actually used anywhere in the codebase. All logging configurations use the log_file_name function from seclab_taskflow_agent.path_utils, which likely manages its own log directory path. Either this variable should be integrated into the actual logging configuration, or it should be removed from the documentation if it's not currently supported.
| LOG_DIR=/app/logs | |
| ``` |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…-dir Add LOG_DIR variable to README
Added LOG_DIR environment variable for logging.